Skip to content

Added Generation V menu sprites#194

Merged
Naramsim merged 5 commits intoPokeAPI:masterfrom
CatKhakis:master
Mar 3, 2026
Merged

Added Generation V menu sprites#194
Naramsim merged 5 commits intoPokeAPI:masterfrom
CatKhakis:master

Conversation

@CatKhakis
Copy link
Contributor

Added generation V menu sprites pulled from Bulbagarden.

  • Sprites have been renamed to match those in generation-v/black-white/
  • Includes both standard and animated sprites under respective directories.
  • Updated README.md to reflect the new directories under generation-v/ along with examples of the standard and animated icons.
  • Same sprites are used in generation III and IV.

@Naramsim
Copy link
Member

Hi! Does it make sense to put them into the folder: sprites/pokemon/versions/generation-v/black-and-white/icons/animated/ ?

Now you put them in: sprites/pokemon/versions/generation-v/animated-icons/

@CatKhakis
Copy link
Contributor Author

CatKhakis commented Feb 13, 2026

I'd argue the following makes more sense as it matches the structure of generation-vii and viii (albeit they do not include animated icons).
-\ generation-v
----\ black-and-white
----\ icons
-------\animated

Rather than the proposed:
-\ generation-v
----\ black-and-white
----\ icons
-------\animated

I have updated the PR to match the first option, but if requested I can have it match the second.

@Naramsim
Copy link
Member

Alright, let's go with the proposed nesting. Could you remove all the leading 0 from the filenames?

@CatKhakis
Copy link
Contributor Author

Leading zeros have been removed!

Copy link
Member

@Naramsim Naramsim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good overall, just some documentation changes are needed to be looked at.

@Naramsim Naramsim merged commit 6881e58 into PokeAPI:master Mar 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants